When the loop fails its i < 48 condition, it will access out of array bounds on GetWeapon right after. This was mostly benign because CHandle m_hMyWeapons[48] is followed by CHandle m_hActiveWeapon in memory, so m_hMyWeapons[48] would access m_hActiveWeapon, which would always be null because the entity can't have an active weapon with no weapons.

PR Checklist

• My PR follows all guidelines in the CONTRIBUTING.md file
• My PR targets a develop branch OR targets another branch with a specific goal in mind